<html>
<head><meta charset="utf-8"><title>get rid of &quot;optimize&quot; option for &quot;rust&quot; in config.toml? · t-compiler · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/index.html">t-compiler</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html">get rid of &quot;optimize&quot; option for &quot;rust&quot; in config.toml?</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="178043065"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178043065" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Guanqun <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178043065">(Oct 13 2019 at 15:47)</a>:</h4>
<p>based on the discussion on this issue: <a href="https://github.com/rust-lang/rust/issues/65352" target="_blank" title="https://github.com/rust-lang/rust/issues/65352">https://github.com/rust-lang/rust/issues/65352</a> do you think it's acceptable to get rid of this option totally? if you agree this is a good idea, I can send the pull request.</p>



<a name="178043130"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178043130" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Guanqun <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178043130">(Oct 13 2019 at 15:48)</a>:</h4>
<p><span class="user-mention" data-user-id="116122">@simulacrum</span></p>



<a name="178043141"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178043141" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Guanqun <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178043141">(Oct 13 2019 at 15:49)</a>:</h4>
<p>I did a quick check and it seems it''s not hard to do this.</p>



<a name="178043269"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178043269" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178043269">(Oct 13 2019 at 15:52)</a>:</h4>
<p>Yeah, sure! I would be happy to review and approve that, I think we might want to just remove it from the documented options as it might be useful to have for internal experimentation</p>



<a name="178043274"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178043274" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178043274">(Oct 13 2019 at 15:52)</a>:</h4>
<p>So basically just drop it from the example config.toml</p>



<a name="178109340"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/131828-t-compiler/topic/get%20rid%20of%20%22optimize%22%20option%20for%20%22rust%22%20in%20config.toml%3F/near/178109340" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Guanqun <a href="https://rust-lang.github.io/zulip_archive/stream/131828-t-compiler/topic/get.20rid.20of.20.22optimize.22.20option.20for.20.22rust.22.20in.20config.2Etoml.3F.html#178109340">(Oct 14 2019 at 14:47)</a>:</h4>
<p>as a followup, the PR is created here: <a href="https://github.com/rust-lang/rust/pull/65408" target="_blank" title="https://github.com/rust-lang/rust/pull/65408">https://github.com/rust-lang/rust/pull/65408</a> Could you please take a look?</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>